Skip to content

fix: Add FIOM bit to senvcfg and henvcfg - #70

Closed
CyrilKoe wants to merge 20 commits into
ez/pulp-v3from
ck/pulp-v3
Closed

fix: Add FIOM bit to senvcfg and henvcfg#70
CyrilKoe wants to merge 20 commits into
ez/pulp-v3from
ck/pulp-v3

Conversation

@CyrilKoe

Copy link
Copy Markdown

Content:

menvcfg, senvcfg, and henvcfg all have the FIOM field; the regfile incorrectly had this flag for machine mode only.

Additional tests:

Boot pipeline with hpd wb cache
https://iis-git.ee.ethz.ch/github-mirror/cheshire/-/pipelines/110639
Boot pipeline with wb cache
https://iis-git.ee.ethz.ch/github-mirror/cheshire/-/pipelines/110638

niwis and others added 20 commits March 30, 2026 14:16
Signed-off-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
Modelsim inference bug. Observed on Questasim 2022.3 during vopt -O5.
Monitor in future releases.

Co-authored-by: Enrico Zelioli <ezelioli@iis.ee.ethz.ch>
* Remove `translate_off` directive from tc_sram_wrapper module (#22).
* Replace sram by sram_pulp: does not introduce cuts and allows for parametric byte width.
* Add HPDCache tc_sram wrappers.
* Add stub FPGA-specific memories: PULP CVA6 does *not* support the `FpgaEn` config option,
  as we want FPGA mappings to emulate taped-out designs as closely as possible.

Co-authored-by: Paul Scheffler <paulsc@iis.ee.ethz.ch>
Co-authored-by: Luca Valente <68963253+luca-valente@users.noreply.github.com>
This commit disables X interface (i.e. sets the CvxifEn parameter to 0)
in all CVA6 configurations relevant to PULP projects.
If the X interface enabled and not used, it forwards illegal
instructions to a co-processor via the core X interface,
This leads to a stall if the X-interface is not connected to an actual
co-processor.
See `cvxif_compressed_if_driver` in `id_stage.sv`.
Co-authored-by: Paul Scheffler <paulsc@iis.ee.ethz.ch>
Co-authored-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
* Bump FPU to pulp-v0.1.1
* Remove vendor folder in CVFPU
* Update Bender.lock
* cvfpu: Exclude util directory
* Bender.lock: Revert unrelated changes

Co-authored-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
Co-authored-by: Enrico Zelioli <ezelioli@iis.ee.ethz.ch>
Experimental feature for some security research.

Signed-off-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
Co-authored-by: aottaviano <aottaviano@iis.ee.ethz.ch>
Co-authored-by: bluew <bluewww@users.noreply.github.com>
Co-authored-by: Enrico Zelioli <enricozelioli97@gmail.com>
Signed-off-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
Co-authored-by: msc22h2 <s290705@studenti.polito.it>
Signed-off-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
Completes support for self-invalidation coherence.
* std_cache: Pack valid/dirty SRAM
* std_cache: Decouple Valid/Dirty WE from Data WE

Co-authored-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
* Add Cheshire integration tests.
* Add PULP Gitlab CI.
* Track Bender.lock file.
* Deactivate 'stale' workflow.
* Do not trigger RISC-V tests on push.
* Report formatting issues via github annotations.
* Deactivate verible check for external repos due to security
concerns (`pull_request_target`)

Co-authored-by: Enrico Zelioli <ezelioli@iis.ee.ethz.ch>
Signed-off-by: Nils Wistoff <nwistoff@iis.ee.ethz.ch>
@CyrilKoe
CyrilKoe requested a review from ezelioli May 10, 2026 21:30
@CyrilKoe CyrilKoe self-assigned this May 10, 2026
@CyrilKoe CyrilKoe added the bug Something isn't working label May 10, 2026
@CyrilKoe
CyrilKoe marked this pull request as ready for review May 11, 2026 04:55
@CyrilKoe
CyrilKoe requested a review from niwis as a code owner May 11, 2026 04:55
@ezelioli

Copy link
Copy Markdown
Collaborator

Done :)

@ezelioli ezelioli closed this May 13, 2026
@CyrilKoe
CyrilKoe deleted the ck/pulp-v3 branch May 13, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants